Skip to content

feat: add webhook endpoints manager#24

Open
tlarrain wants to merge 1 commit into
mainfrom
onb-2040-agregar-cambios-en-sdk-de-ruby
Open

feat: add webhook endpoints manager#24
tlarrain wants to merge 1 commit into
mainfrom
onb-2040-agregar-cambios-en-sdk-de-ruby

Conversation

@tlarrain
Copy link
Copy Markdown

Contexto

Vamos a agregar un endpoint nuevo a la API y necesitamos que el SDK se actualice

¿Qué hay de nuevo?

  • Webhook endpoints manager

Tests

  • Tests unitarios

Safety Checks

  • Versión y CHANGELOG actualizados (si corresponde)

@tlarrain tlarrain requested a review from pibahamondesw May 15, 2026 19:46
Copy link
Copy Markdown
Contributor

@pibahamondesw pibahamondesw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Creo que todo debería vivir en v2, no v1, que no deberíamos seguir manteniendo ni extendiendo. Imagino que hay que cambiar sintaxis de los llamados y esas cosas así que no revisé nada más, resolicita no más cuando lo cambies.

@tlarrain
Copy link
Copy Markdown
Author

Creo que todo debería vivir en v2, no v1, que no deberíamos seguir manteniendo ni extendiendo. Imagino que hay que cambiar sintaxis de los llamados y esas cosas así que no revisé nada más, resolicita no más cuando lo cambies.

@pibahamondesw los endpoints que le pega son /v1/webhook_endpoints/. Por lo que entendí de esta librería, esa separación es consistente con las versiones de la API o no?

@pibahamondesw
Copy link
Copy Markdown
Contributor

@pibahamondesw los endpoints que le pega son /v1/webhook_endpoints/. Por lo que entendí de esta librería, esa separación es consistente con las versiones de la API o no?

Uhh no recuerdo, pensaba que era según la versión de fintoc-ruby. Reviso y te confirmo altiro

@pibahamondesw
Copy link
Copy Markdown
Contributor

@tlarrain efectivamente era como dices, reviso altiro jaja 😭

Copy link
Copy Markdown
Contributor

@pibahamondesw pibahamondesw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Faltaría agregar la versión y el changelog para que puedas sacar el release

@tlarrain
Copy link
Copy Markdown
Author

LGTM! Faltaría agregar la versión y el changelog para que puedas sacar el release

@pibahamondesw DX está trabajando en github actions para hacer los releases! No había cachado que este repo tiene changelog (fintoc-node y fintoc-python no tienen), pero la idea es crear tags en github mismo con el changelog al momento de hacer el bump (que también lo haría la action de una). Te tinca esperar a ver cómo queda eso? Quizás el changelog va a quedar obsoleto para dar paso a las release-notes del tag

Copy link
Copy Markdown
Contributor

@pibahamondesw pibahamondesw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! No cachaba eso

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants